Don't mix GJS and GObject signal systems

GJS implements a basic signal system that allows monkey-patching
JS objects with signal methods resembling the GObject ones. However
it's clearly not a good idea to replace the actual GObject methods,
so use the proper GObject facilities when inheriting from GObject.

https://bugzilla.gnome.org/show_bug.cgi?id=778660

network: Ensure the connection list is sorted after rename

Items were inserted correctly but the synchronisation was lost if the
name of a connection was changed. Simply making sure the position is
correct after a connection is updated fixes the issue.

Reported-by: Oliver Haessler <ohaessle@redhat.com>
https://bugzilla.gnome.org/show_bug.cgi?id=778686

ui: Allow moving menu items to a certain position

This function is a helper to simplify keeping menu items ordered when
their order is updated on the fly (e.g. network connections being
renamed).

https://bugzilla.gnome.org/show_bug.cgi?id=778686

extensionDownloader: return tuple of string on success

Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=777544

portalHelper: Don't fail to load because of TLS errors

Accept self-signed and expired certificates as if we errored out, we
couldn't access the network at all. Consider this insecure though.

https://bugzilla.gnome.org/show_bug.cgi?id=778253

Updated Ukrainian translation

tests: require given version of Gtk

tests: load required version of Clutter/Gtk early

Updated Spanish translation

Use the versioned libmutter*.so versions

Mutter now provides versioned libraries and pkg-config files, meaning
an application using libmutter and friends need to depend on a specific
version of the API.

https://bugzilla.gnome.org/show_bug.cgi?id=777317

loginDialog: don't allow type ahead at the login screen

It's weird if after you select a username from the user list, there's
a password already filled in.

This commit disables at that feature for the login screen
(but keeps it in tact for the unlock screen)

https://bugzilla.gnome.org/show_bug.cgi?id=766139

Updated Czech translation

Updated Norwegian bokmål translation.

Update Indonesian translation

portalHelper: Fix signal arguments

How did this work in tests?!

https://bugzilla.gnome.org/show_bug.cgi?id=778552

portalHelper: Fix runtime warning

When there is no URI, then the title should be empty, not null:
Gjs-WARNING **: JS ERROR: Error: Argument 'str' (type utf8) may not be null

https://bugzilla.gnome.org/show_bug.cgi?id=778552

portalHelper: Add security icon to titlebar

This adds a security icon (either secure or insecure) to the portal
helper's title bar. As soon as a part or all of the page and its content
is served insecurely, the icon shown will be a broken padlock.

https://bugzilla.gnome.org/show_bug.cgi?id=749197

Updated Czech translation

windowManager: Use g-s-d wacom dbus helpers to update pad leds/oleds

This keeps feedback on the device itself in sync with the overall pad
state, a feature we had previously through g-s-d.

https://bugzilla.gnome.org/show_bug.cgi?id=776543

windowManager: Handle MetaDisplay::show-osd signal

Propagate as-is to the OsdWindowManager.

https://bugzilla.gnome.org/show_bug.cgi?id=771098

js: Avoid double declarations with let

The following code is a syntax error in ES6:

    let a = 'something';
    let a = 'other thing';

Previously GJS would silently accept this code, but in the next release the
SpiderMonkey JS engine will be more ES6-compliant.

https://bugzilla.gnome.org/show_bug.cgi?id=778425

ibusManager: Use const correctly

Per ES6, a variable declared const should only be valid inside its lexical
scope. Previously, GJS would accept this code, but that will change in the
SpiderMonkey JS engine in the next release of GJS.

https://bugzilla.gnome.org/show_bug.cgi?id=778425

main: Destroy GjsContext before exit

Here are a few places where I missed it before: the GjsContext must be
unrefed before exiting, or it will crash on exit.

https://bugzilla.gnome.org/show_bug.cgi?id=778425

Use /proc/self/cmdline on Solaris as well in shell_global_reexec_self()

Solaris 11.3.5 and later have a Linux-compatible implementation of
/proc/self/cmdline, so the code to use it can be enabled in gnome-shell
on Solaris - if used on an older OS, it simply fails to open the file
and returns without doing anything, just as the code did before enabling
this.

https://bugzilla.gnome.org/show_bug.cgi?id=776199

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

widget: Only include visible actors in focus chain

It isn't useful to move the keyboard focus to a hidden actor, so
only include visible actors in the focus chain - this is in fact
the documented behavior of st_widget_get_focus_chain(), so having
the default implementation return all children has always been
unexpected.

https://bugzilla.gnome.org/show_bug.cgi?id=778158

st-theme-node: Fix lookup of time values

We have a time when the content type is seconds *or* milliseconds,
not seconds *and* milliseconds. Whoops ...

https://bugzilla.gnome.org/show_bug.cgi?id=778145

AppFavorites: order alphabetically by key

Update zh_CN translation

Update Finnish translation

Make all our window clones use the MetaWindowActor as source

Using a MetaWindowActor's shaped texture as the source for window
clones means that if there are further MetaSurfaceActor children
(e.g. a wayland client using sub-surfaces) they don't get cloned.

This obviously wasn't an issue until wayland clients introduced the
possibility of having multiple MetaSurfaceActors under a
MetaWindowActor but there's no fundamental reason we can't clone the
toplevel actor.

WorkspaceThumbnail.WindowClone is the one class that was already using
the MetaWindowActor instead of the texture although it seems to have
been an unintended change in commit
8b99617513da1713ff88ca4abf54e6125405d31c.

https://bugzilla.gnome.org/show_bug.cgi?id=756715

Update Belarusian translation

background: Disconnect from 'prepare-for-sleep' signal

This signal connection was keeping Background instances from being
garbage collected.

https://bugzilla.gnome.org/show_bug.cgi?id=777934

background: Remove a couple of unused variables

Nothing uses these.

https://bugzilla.gnome.org/show_bug.cgi?id=777934

Update Polish translation

Fix a translator comment

They need to be exactly one line above a string to show up in .po files.

windowManager: Avoid fullscreen animation if the window has no texture

There is hardly anything to animate, so just avoid the animation in this
case.

https://bugzilla.gnome.org/show_bug.cgi?id=777784

windowManager: Avoid reentrancy on the ::size-changed handler

We have no guarantees on the number of ::size-changed signals that we may
receive, so the _sizeChangedWindow handler may run multiple times, which
leads to multiple calls to meta_plugin_size_change_completed(). So double
check the actor is not already being animated in the _sizeChangedWindow
handler to avoid reentrancy.

https://bugzilla.gnome.org/show_bug.cgi?id=777784

Update Kazakh translation

Update Slovak translation

build: Remove C++ check

As it's not used any more since commit
ed99bef45841c407683cd7d353f0e2d364977261.

https://bugzilla.gnome.org/show_bug.cgi?id=762444

portalHelper: Don't change the main window title

The title of the window should not be in control of a potentially
hostile hotspot provider, so only set the subtitle to be that of the
page, the main title will stay the same.

The subtitle will also be set to a URI, so that the hotspot cannot be
used to control the title shown in our UI.

Helps https://bugzilla.gnome.org/show_bug.cgi?id=749197

portalHelper: Set a minimum window size

Even though the window is maximised when created, the window is still
resizable. This avoids making the window smaller than usable. If the
screen is smaller than the minimum window size, the maximise should make
it fullscreen and non-resizable.

https://bugzilla.gnome.org/show_bug.cgi?id=735233

portalHelper: Use private data/cache directories

https://bugzilla.gnome.org/show_bug.cgi?id=775639

Properly detect changes in .desktop files

This fixes the shell not picking updates to the contents of the underlying
.desktop files for application launchers, or when a .desktop file is
overriden by a user-installed one under ~/.local/share/applications

https://bugzilla.gnome.org/show_bug.cgi?id=773636

Update French translation
(cherry picked from commit 3c6c51125ced82f58977eef84d22f769ef754c21)

Update Polish translation

ibusCandidatePopup: handle mouse scroll event on candidates

When use mouse scroll button on input method candidates,
move the cursor up/down on candidates.

https://bugzilla.gnome.org/show_bug.cgi?id=776032

portalHelper: Handle Ctrl+W/Ctrl+Q to close window

https://bugzilla.gnome.org/show_bug.cgi?id=764133

Update Slovak translation

Update Catalan translation

Update Brazilian Portuguese translation

Fix Icon filename in translations

Revert "objectManager: handle proxies coming and going"

This reverts commit 1ef626213952ea8568bb35ec0e96788a696d1fee.

https://bugzilla.gnome.org/show_bug.cgi?id=772589

Update Slovak translation

objectManager: handle proxies coming and going

Ever since commit b8e29ae8c78658917c8a8ddd391667d7d9e36971
(I think), start up is littered with this message:

 Gjs-WARNING **: JS ERROR: could not get remote objects for service
 org.gnome.SettingsDaemon.Smartcard path

since gnome-shell is now started before gnome-settings-daemon.

This commit addresses the problem by making the object manager code
not try to autostart its proxy, and instead wait for it to appear.

https://bugzilla.gnome.org/show_bug.cgi?id=772589

network: Avoid JS error when activating VPN connection

If the call to settings.get_connection_by_path in
ensureActiveConnectionProps returns null, we'll hit a JS error here.
Seems to happen always when activating a VPN connection. Avoid that.

Giovanni says:

"I believe this is papering over an existing bug, but it's possible for
settings.get_connection_by_path() to legitimately return null (if the
connection is owned by a different user and invisible to the current
one), so the fix is correct anyway."

https://bugzilla.gnome.org/show_bug.cgi?id=759793

Updated Spanish translation

Update Hungarian translation

Update Hungarian translation

Update Finnish translation

Bump version to 3.23.3

Update NEWS.

gtk-embed: Don't try to compare with unrealized window

After having embedded a Gtk window, we start to listen for the
corresponding MetaWindow to know when it has been mapped by the
compositor. If the embedder does not realize the window immediately,
and some other window is created and mapped before the embedder
realizes the window, we would prior to this patch try to dereference
the not created GdkWindow. Fix this by NULL checking the GdkWindow
before dereferencing. We will never miss the needed MetaWindow creation
since it can not have been mapped have it not yet been realized.

https://bugzilla.gnome.org/show_bug.cgi?id=776130

appFavorites: add Calculator to rename list again

So far it's only the second app to be on the rename list twice, but no
doubt there will soon be more....

tests: Destroy GjsContext before exit

This will be required in the upcoming version of GJS.

https://bugzilla.gnome.org/show_bug.cgi?id=775374

appFavorites: add Epiphany to rename list

Update Italian translation
(cherry picked from commit 473158db8eabe4c74978b9a6c1759a4ae0480ee2)

main: Destroy GjsContext before exit

This will be required in the upcoming version of GJS.

The reference count on ShellGlobal is 2 at this point, because JS holds a
reference due to the "window.global = Shell.Global.get()" line in
ui/environment.js. Therefore, destroy the GjsContext first, then unref
the ShellGlobal object.

Cleaning up ShellGlobal was previously only enabled behind a debug
environment variable, but it should be required now.

https://bugzilla.gnome.org/show_bug.cgi?id=775374

helpers: Destroy GjsContext before exit

This will be required in the upcoming version of GJS.

https://bugzilla.gnome.org/show_bug.cgi?id=775374

appFavorites: add Geary in rename list

Signed-off-by: Niels De Graef <nielsdegraef@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=775507

updated Punjabi Translation

Update Chinese (Taiwan) translation

notificationDaemon: Keep source alive when replacing notification

When a GNotification has the same ID as a previous one, it should
be shown as a new notification after withdrawing the old one.
However for this to work, we must not destroy the corresponding
source if withdrawing the old notification lets the notification
count drop to zero, so make sure the source is kept alive until
the replace operation has completed.

https://bugzilla.gnome.org/show_bug.cgi?id=775149

Update Kazakh translation

NotificationDaemon: create the application proxy asynchronously

Clicking on calendar notifications might block till the DBus request times
out if the application being poked happens to be non-responsive. Perform
this asynchronously so we don't block if that is the case.

Update German translation

Update German translation

Update Turkish translation

Bump version to 3.23.2

Update NEWS.

build: Fix file reference

appDisplay: Fix completion handler for empty animations

If an onComplete handler is passed to animate(), it is set to run at
the end of the animation via the icon grid's ::animation-done signal.
Currently the signal is connected after starting the animation, with
the result that the handler doesn't run when the animation completes
immediately (because there are no icons to animate). Fix this by only
starting the animation after connecting the signal.

https://bugzilla.gnome.org/show_bug.cgi?id=774381

appDisplay: Use correct view for animation

We currently assume that the current view matches the 'app-picker-view'
setting. While that is usually the case, there is one notable exception:
While there isn't sufficient usage data (yet), we show all applications
instead of an empty frequent view regardless of the setting. We should
animate the actually visible icons in that case, not the (non-existent)
ones from the hidden frequent view.

https://bugzilla.gnome.org/show_bug.cgi?id=774381

Update Russian translation
(cherry picked from commit f395d3892c278db216b1b460bd38281bcda80105)

portalHelper: Use a constant for the host name and URI

https://bugzilla.gnome.org/show_bug.cgi?id=769940

portalHelper: Address intermittent TLS failures

Users are currently experiencing intermittent (and hard to debug) TLS
errors when loading the portal authentication page. This could be
caused by using http://www.gnome.org/ for our connectivity checks
because it redirects to HTTPS.

We can completely remove TLS from the equation by not using HTTPS.
http://nmcheck.gnome.org/ is a good choice because it doesn't redirect
to HTTPS and was created for NetworkManager's connectivity check.

https://bugzilla.gnome.org/show_bug.cgi?id=769940

windowManager: Handle fullscreening from zero sized windows

Wayland clients can request their surfaces to be fullscreened before
commiting a buffer which means that we need to handle fullscreen
requests for which the old size is 0x0, preferably without warnings.

Since the mapping animation also runs for these windows, we can simply
bail out and ignore the fullscreen size change.

https://bugzilla.gnome.org/show_bug.cgi?id=770345

windowManager: Fix fullscreen animations of wayland clients

Wayland clients are in control of their window size so the existing
mutter plugin API, which assumes size changes are synchronous, doesn't
work for them since when our size-change handler runs the MetaWindow's
size isn't final yet.

To fix this, the mutter plugin API was extended with a size-changed
vfunc that lets us know when the MetaWindow size has actually
changed. This way we can make the window snapshot and get the old
window size on the existing size-change handler and later, on the new
size-changed handler, get the new size and start the animation.

https://bugzilla.gnome.org/show_bug.cgi?id=770345

shell-wm: Add a size-changed signal to handle the new plugin vfunc

https://bugzilla.gnome.org/show_bug.cgi?id=770345

texture-cache: Warn when loading sliced image fails

Sliced images are loaded into a group actor with one child actor
per slice. In case loading the image fails, we currently quietly
return the empty group actor, which makes diagnosing problems
unnecessarily hard - just be a bit more verbose on failure.

https://bugzilla.gnome.org/show_bug.cgi?id=774805

animations: Guard against empty animations

Our animation code not only relies on the animation being loaded
(which we handle), but also on having at least one frame - otherwise
the computation of the next frame index will turn up NaN through
division by zero. Guard against this case by treating empty animations
as not loaded.

https://bugzilla.gnome.org/show_bug.cgi?id=774805

Updated Norwegian bokmål translation.

Update Friulian translation

Update Finnish translation
(cherry picked from commit d0e811214b60bcbc64261839150e9a597358fdc0)

Update Kazakh translation
(cherry picked from commit 52bbee80a0ccd11c1ed2ebb707cab7ec60804e0a)

Update Brazilian Portuguese translation
(cherry picked from commit 7098d6930fb39658bf32fb6375b9eb7b39dc65bf)

Updated Czech translation

Update Polish translation

runDialog: Mark error message as translatable

https://bugzilla.gnome.org/show_bug.cgi?id=774643

Updated Czech translation

Update Friulian translation

Update Spanish translation